Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(crwa): Fix ts-to-js script #10476

Merged
merged 1 commit into from
Apr 18, 2024
Merged

chore(crwa): Fix ts-to-js script #10476

merged 1 commit into from
Apr 18, 2024

Conversation

Josh-Walker-GM
Copy link
Collaborator

Problem
The ts-to-js script within the create-redwood-app package is failing when executed.

It appears when I upgraded to prettier v3 I failed to consider this particular code. Specifically, prettier v3 changed their API to be async rather than sync so we must now await the function call to obtain the value and not a promise. See: here for prettier's breaking change notice.

Changes

  1. Await the prettier format call.

@Josh-Walker-GM Josh-Walker-GM added release:chore This PR is a chore (means nothing for users) changesets-ok Override the changesets check labels Apr 18, 2024
@Josh-Walker-GM Josh-Walker-GM added this to the chore milestone Apr 18, 2024
@Josh-Walker-GM Josh-Walker-GM self-assigned this Apr 18, 2024
Copy link
Member

@Tobbe Tobbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Tobbe Tobbe merged commit 60798a6 into main Apr 18, 2024
48 of 51 checks passed
@Tobbe Tobbe deleted the jgmw/chore-crwa-ts-to-js-fix branch April 18, 2024 21:07
dac09 added a commit that referenced this pull request Apr 19, 2024
…g-gen-mw-p2

* 'main' of github.com:redwoodjs/redwood:
  chore(tests): add use location typetests to prevent regressions (#10474)
  feat: Remove `--performance` option from `yarn rw build` (#10453)
  fix(cli): await async prettier functions (#10478)
  chore(crwa): Fix `ts-to-js` script (#10476)
Josh-Walker-GM added a commit that referenced this pull request Apr 19, 2024
@Josh-Walker-GM Josh-Walker-GM modified the milestones: chore, v8.0.0 Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants